Skip to content

Conversation

@nordicjm
Copy link
Contributor

@nordicjm nordicjm commented Nov 2, 2021

This addresses USB issues with the BL654 USB dongle which prevent USB
from working properly (e.g. hci_usb) or do not automatically give
access to the console of the board (most other samples) by adding a
feature that will automatically enable the USB peripheral on startup
without needing to be manually added to sample/user applications.

Signed-off-by: Jamie McCrae [email protected]

Fixes #39904 and addresses #39969

This addresses USB issues with the BL654 USB dongle which prevent USB
from working properly (e.g. hci_usb) or do not automatically give
access to the console of the board (most other samples) by adding a
feature that will automatically enable the USB peripheral on startup
without needing to be manually added to sample/user applications.

Signed-off-by: Jamie McCrae <[email protected]>

config USB_UART_CONSOLE
default y
default y if !USB_DEVICE_BLUETOOTH
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes in this file should be in own commit, which fixes #39904.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lairdjm could you please do that as suggested by @jfischer-no?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@carlescufi this is awaiting a USB fix from @jfischer-no

Copy link
Contributor

@jfischer-no jfischer-no Nov 24, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For completeness, option USB_UART_CONSOLE is removed, see #40428. Shield approach in #40220 was not accepted. I opened #40645 to continue the discussion.

Anyway, I do not want to accept any USB device support setting at board level anymore. The boards in Zephyr tree should be in minimal configuration, https://docs.zephyrproject.org/latest/guides/porting/board_porting.html#general-recommendations, so no configuration to enable USB support or USB device classes.

@jfischer-no
Copy link
Contributor

Apart from #39987 (comment), this patch is a selfish approach, which will also be a bad example for other boards. I am looking at how we can make it better and bring USB device support initialization during startup back in case only CDC ACM class is enabled and CDC ACM UART is used as logging/shell backend.

@nordicjm
Copy link
Contributor Author

@carlescufi can we progress with this temporary fix for the upcoming release until there is a proper global fix for this? As this basically means the bl654_usb board is dead without manual changes to customer codebases

@nordicjm
Copy link
Contributor Author

Fixed in #42371

@nordicjm nordicjm closed this Mar 16, 2022
@nordicjm nordicjm deleted the bl654usbusbfix branch March 18, 2022 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bl654_usb does not work with hci_usb sample application

5 participants